From: Owen W. Taylor <otaylor@fishsoup.net>
Date: Thu, 25 Sep 2014 06:34:30 +0000 (-0400)
Subject: --help should always go to stdout
X-Git-Tag: archive/raspbian/2022.1-3+rpi1~1^2~1107
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=3400f2d2aef70ae9abc319ddb73ed407befbe528;p=ostree.git

--help should always go to stdout

The standard convention is that the  output of --help should go to standard
output (so that it can be piped to a pager and searched.) See, e.g., the
GNU coding standards.

https://bugzilla.gnome.org/show_bug.cgi?id=737194
---

diff --git a/src/ostree/ot-admin-builtin-instutil.c b/src/ostree/ot-admin-builtin-instutil.c
index c87ffa4e..a0a29d28 100644
--- a/src/ostree/ot-admin-builtin-instutil.c
+++ b/src/ostree/ot-admin-builtin-instutil.c
@@ -119,12 +119,14 @@ ot_admin_builtin_instutil (int argc, char **argv, OstreeSysroot *sysroot, GCance
 
   if (subcommand_name == NULL)
     {
+      void (*print_func) (const gchar *format, ...) = want_help ? g_print : g_printerr;
+
       subcommand = admin_instutil_subcommands;
-      g_print ("usage: ostree admin instutil COMMAND [options]\n");
-      g_print ("Builtin commands:\n");
+      print_func ("usage: ostree admin instutil COMMAND [options]\n");
+      print_func ("Builtin commands:\n");
       while (subcommand->name)
         {
-          g_print ("  %s\n", subcommand->name);
+          print_func ("  %s\n", subcommand->name);
           subcommand++;
         }
 
diff --git a/src/ostree/ot-builtin-admin.c b/src/ostree/ot-builtin-admin.c
index ef54797e..afc3bb33 100644
--- a/src/ostree/ot-builtin-admin.c
+++ b/src/ostree/ot-builtin-admin.c
@@ -147,12 +147,14 @@ ostree_builtin_admin (int argc, char **argv, OstreeRepo *repo, GCancellable *can
 
   if (subcommand_name == NULL)
     {
+      void (*print_func) (const gchar *format, ...) = want_help ? g_print : g_printerr;
+
       subcommand = admin_subcommands;
-      g_print ("usage: ostree admin --sysroot=PATH COMMAND [options]\n");
-      g_print ("Builtin commands:\n");
+      print_func ("usage: ostree admin --sysroot=PATH COMMAND [options]\n");
+      print_func ("Builtin commands:\n");
       while (subcommand->name)
         {
-          g_print ("  %s\n", subcommand->name);
+          print_func ("  %s\n", subcommand->name);
           subcommand++;
         }
 
diff --git a/src/ostree/ot-main.c b/src/ostree/ot-main.c
index 6adb2d22..585e44f7 100644
--- a/src/ostree/ot-main.c
+++ b/src/ostree/ot-main.c
@@ -201,7 +201,7 @@ ostree_run (int    argc,
           g_set_error_literal (&error, G_IO_ERROR, G_IO_ERROR_FAILED,
                                "No command specified");
         }
-      ostree_usage (argv, commands, TRUE);
+      ostree_usage (argv, commands, !want_help);
       goto out;
     }